Skip to content

Remove unused scripts and things#5038

Merged
zachmargolis merged 5 commits intomainfrom
margolis-remove-unused
May 7, 2021
Merged

Remove unused scripts and things#5038
zachmargolis merged 5 commits intomainfrom
margolis-remove-unused

Conversation

@zachmargolis
Copy link
Copy Markdown
Contributor

A week late, but I wanted to join in on the :delete-all-the-things: party

Comment thread bin/generate_saml_pki
@@ -1,2 +0,0 @@
#!/bin/sh
openssl req -x509 -sha256 -days 365 -newkey rsa:2048 -keyout keys/saml.key.enc.example -out certs/saml.crt.example -config config/openssl.conf
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread Makefile
find ./app/javascript -name "*.js*" | xargs ./scripts/check-assets

generate_deploy_checklist:
ruby lib/release_management/generate_deploy_checklist.rb
Copy link
Copy Markdown
Contributor Author

@zachmargolis zachmargolis May 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread bin/fetch_github_metrics

require_relative '../lib/github_metrics'

GithubMetrics::PrAgeReporter.new.call
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we use this for sprints anymore?

Copy link
Copy Markdown
Contributor

@orenyk orenyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me!

@zachmargolis zachmargolis merged commit 69f5d5c into main May 7, 2021
@zachmargolis zachmargolis deleted the margolis-remove-unused branch May 7, 2021 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants